-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create login page #1394
base: ui/redesign
Are you sure you want to change the base?
Create login page #1394
Conversation
…d the widgets positions. Co-authored by: Granja5 <[email protected]>
textSelectionTheme: const TextSelectionThemeData( | ||
cursorColor: Colors.white, | ||
selectionHandleColor: Colors.white, | ||
data: applicationLightTheme.copyWith( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if this must be inserted into the theme directly, once it is finished.
fromHeroContext, | ||
toHeroContext, | ||
) { | ||
return ScaleTransition( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the purpose of this transition? Can you upload a "demo"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I sent it as a video in the issue page, but here it is:
https://github.com/user-attachments/assets/12611925-2dc3-45bb-b13a-136a582c9b23
As there's not a splash screen yet, there's no Hero linking transition widget :/
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## ui/redesign #1394 +/- ##
===========================================
- Coverage 12% 12% -0%
===========================================
Files 265 270 +5
Lines 7209 7286 +77
===========================================
+ Hits 806 808 +2
- Misses 6403 6478 +75 |
I implemented the splash screen without the "another_flutter_splash_screen" package. |
…sh screen still exists and appears after the native one. flutter_native_splash package was added to the project.
…into separate widget classes in the widget folder for better modularity and code organization.
…cified that only underlined text activates the button.
…when the language is changed
… The animation in the logo now works flawlessly.
|
@pedroafmonteiro if |
…s not being used.
If it's already ready to review, just convert the PR to "already to review". I am excited to look into this. |
The login with credentials popup is the only thing left. It should be done between today and tomorrow! |
Closes #1375
To implement the splash screen
To add the gradient and animation to the splash screen I needed to add this package, because it allow us to create a splash screen with custom widgets for the gradients and the animation: https://pub.dev/packages/another_flutter_splash_screenThis was accomplished without adding the package.The native splash screen was replaced by the same background color as the new splash and login page with a transparent logo image through this package. I think that, before, both the android and ios splash screens were implemented manually, but this package does that automatically with a simple configuration in the pubspec.yaml file: https://pub.dev/packages/flutter_native_splash
Review checklist
whatsnew/whatsnew-pt-PT
changelog.md
with the change